-
-
Notifications
You must be signed in to change notification settings - Fork 38
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Issue 1195 - Move Copy to Clipboard to a function to reduce code duplication #1218
Conversation
@2Abendsegler schaust du bitte nochmal drüber. Ich habe die Funtion angelegt und die 3 Stellen ausgetauscht, ich hoffe ich hab keinen Copy Link vergessen. |
Hab heute keine Lust, das Teil muss noch warten. 🙁 |
Ich kenne auch nur die drei Copybuttons. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Meine Fresse, da muss man ja studiert haben um das zu verstehen. 😂
Das kann ich nicht mal so zwischendurch machen. Ich versuchs morgen oder heute Abend.
Die function gehört eigentlich weiter runter in den Bereich // GClh Functions
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Die GC Codes funktionieren prima.
Zu den Koordinaten:
Wenn GCTour im Einsatz ist und mit GCTour Koordinaten verschoben wurden dann werden diese nicht berücksichtigt. Meines Erachtens hängt das damit zusammen, dass die Ermittlung der Koordinaten nicht erst beim Klick durchgeführt wird. Die GCTour Daten sind zu diesem Zeitpunkt noch gar nicht vorhanden. Die Ermittlung sollte nach meinem Verständnis immer erst dann erfolgen, wenn der Button betätigt wird, auch für den GC Code, nur macht es dort keinen Unterschied.
(Hier wird auch deutlich dass der Button rechts "Copy Data to Clipboard" auch noch nicht sauber funktioniert. Auch hier darf der Aufbau der Punkte im Dropdown erst stattfinden wenn man mit der Maus darüber geht. Und er muss auch immer wieder erneut stattfinden, weil sich zwischenzeitlich die GCTour Koordinaten geändert haben könnten. Hier werden zwar die richtigen Koordinaten ermittelt, es wird aber nicht bemerkt, dass es keine geänderten Koordinaten mehr gibt und deshalb der Punkt aus dem Dropdown entfernt werden müßte.
Dazu mache ich aber ein eigenes Issue auf. Ich werde mich darum kümmern.)
Ich glaube ich verstehe das Problem. Ich werd mir mal GCTour installieren und dann schauen was genau du meinst. |
Eigentlich ist es nicht notwendig GCTour zu installieren. Es muss lediglich sichergestellt werden, dass die Koordinaten erst beim Klick ermittelt werden. Ich vermute die GCTour Koordinaten entstammen noch einer Zeit in denen es mit geänderten Koordinaten im Cache nicht so weit her war. Vermutlich könnte GCTour heute auch mit den geänderten Cache Koordinaten arbeiten. Ich nutze die Koordinaten von GCTour ständig. |
Ich hab die Funktion zum Kopieren der Koordinaten wieder raus genommen bei mir. Außerdem ist die ctoc funktion nun an der richtigen stelle im Coding. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Aus meiner Sicht prima.
Allerdings gibt es scheinbar ne Überschneidung im Coding zu Copy coordinates to clipboard
.
Wir können aber trotzdem mergen und ich schieb die Teile nochmal nach.
Soll ich mergen und nachschieben?
Habe die Konflikte noch beseitigt. |
Du hättest auch gern mergen können 😃 |
No description provided.